Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 2.4 (Desktop) #290

Merged
merged 92 commits into from
Jan 10, 2024
Merged

Update 2.4 (Desktop) #290

merged 92 commits into from
Jan 10, 2024

Conversation

floribe2000
Copy link
Member

No description provided.

iTTou and others added 30 commits November 19, 2023 10:51
This change improves user experience by allowing quick sharing of images. However, as this feature is not supported in all browsers, an error message has been added for unsupported cases.
Deleted the BuildShareResult.cs file and updated handling of build sharing, image saving and link generation in ShipStatsTab. Also user notifications.
also fix default starting folder for the folder picker being a collection instead of an actual folder.
floribe2000 and others added 17 commits January 2, 2024 16:14
* add service to navigate from one page of the app to the other without losing builds.
add buttons to go back to ships stats page in all app pages

* integrate metrics logging into the service

* fix multiple blank lines

* fix wrong localization

* Simplify the navigator service and drastically reduce methods overloading

* remove unnecessary service injections

* make url generation method static

* add documentation

* change container lists into an immutable one

* move DataTransfer folder from Infrastructure to features

* rearrange DataTransfer directory and split navigation-related classes into new Navigation feature

* remove metricLabel parameter from AppNavigator service methods and integrate them into LogMetrics method logic

* move updateBuildConfirmationDialog into its own method

---------

Co-authored-by: floribe2000 <git@floribe2000.de>
* add parameterized localization support for ILocalizer

* apply new parameterized localization to OverwriteExistingBuildConfirmationDialog and rename the dialog to better show its use case

* remove methods for parameterized game localization

game localization is not localized, only app localization can use placeholders
* New translations translation.resx (French)

* New translations translation.resx (Spanish)

* New translations translation.resx (German)

* New translations translation.resx (Hungarian)

* New translations translation.resx (Italian)

* New translations translation.resx (Japanese)

* New translations translation.resx (Korean)

* New translations translation.resx (Dutch)

* New translations translation.resx (Polish)

* New translations translation.resx (Russian)

* New translations translation.resx (Turkish)

* New translations translation.resx (Chinese Simplified)

* New translations translation.resx (Chinese Traditional)

* New translations translation.resx (Portuguese, Brazilian)

* New translations translation.resx (Russian)

* New translations translation.resx (Japanese)

* New translations translation.resx (Chinese Traditional)
@floribe2000 floribe2000 marked this pull request as ready for review January 6, 2024 22:51
the source generator tests do not work properly with the new nunit version
yesod30 and others added 5 commits January 9, 2024 20:14
* first draft of a custom pager

* Add page size to app settings

* remove unnecessary custom pager component

* add missing localization

* increase default page size from 25 to 50

* change pageSize url parameter setting threshold

* remove pageSize property direct set and delegate its setting to SetRowsPerPageAsync method of the MudDataGrid component

* remove unnecessary pageCounter variable
Copy link

sonarcloud bot commented Jan 9, 2024

@floribe2000 floribe2000 merged commit 32abd2c into release/desktop Jan 10, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants